Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only use ci-friendly version if there is such property defined #3320

Merged

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Jan 5, 2024

Currently it can happen that one uses one of the "trigger properties" that enable Tycho pomless feature to assume that a cifriendly version is to be used even though ci-friendly versions are actually not used at all.

This tries to differentiate the two cases by checking if there is actually a property defined that could be used in a ci-friendly way.

@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Jan 5, 2024
@laeubi laeubi force-pushed the only_use_ci_firendly_when_in_model branch from 2b9de3a to 70dd0da Compare January 5, 2024 14:37
@laeubi laeubi force-pushed the only_use_ci_firendly_when_in_model branch from 70dd0da to 9352449 Compare January 5, 2024 15:42
Copy link

github-actions bot commented Jan 5, 2024

Test Results

  576 files  ±0    576 suites  ±0   3h 51m 14s ⏱️ - 1m 42s
  380 tests ±0    374 ✅ ±0   6 💤 ±0  0 ❌ ±0 
1 140 runs  ±0  1 121 ✅ ±0  19 💤 ±0  0 ❌ ±0 

Results for commit fc0c09b. ± Comparison against base commit 64b3ccf.

♻️ This comment has been updated with latest results.

Currently it can happen that one uses one of the "trigger properties"
that enable Tycho pomless feature to assume that a cifriendly version is
to be used even though ci-friendly versions are actually not used at
all.

This tries to differentiate the two cases by checking if there is
actually a property defined that could be used in a ci-friendly way.
@laeubi laeubi force-pushed the only_use_ci_firendly_when_in_model branch from 9352449 to fc0c09b Compare January 5, 2024 17:32
@laeubi laeubi merged commit 6b1c316 into eclipse-tycho:main Jan 6, 2024
10 checks passed
Copy link

github-actions bot commented Jan 6, 2024

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@laeubi laeubi added the sponsored The bugfix or feature was gently sponsored by an individual or company label Jan 9, 2024
@laeubi
Copy link
Member Author

laeubi commented Jan 9, 2024

This bugfix was gently sponsored by Sigasi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change sponsored The bugfix or feature was gently sponsored by an individual or company
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants